Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Additional JWT Security Token Support #1337

Merged
merged 1 commit into from Oct 29, 2014
Merged

Additional JWT Security Token Support #1337

merged 1 commit into from Oct 29, 2014

Conversation

RyanMelenaNoesis
Copy link
Contributor

Updates to support for JWT based authentication (primarily around JWT decoding/decryption and claims checking)

@monoadmin
Copy link

Hello! I'm the build bot for the Mono project. I need approval from a Mono team member to build this pull request. A team member should reply with "approve" to approve a build of this pull request, "whitelist" to whitelist this and all future pull requests from this contributor, or "build" to explicitly request a build, even if one has already been done. Contributors can ignore this message.

RyanMelenaNoesis
RyanMelenaNoesis reviewed Oct 14, 2014
View reviewed changes
mcs/class/System.IdentityModel/System.IdentityModel.Tokens/SecurityTokenExpiredException.cs

namespace System.IdentityModel.Tokens
{
public class SecurityTokenExpiredException : SecurityTokenValidationException
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Implement missing exception

@migueldeicaza
Copy link
Contributor

Please notify mono-deve-list and discuss this PR there.

migueldeicaza
migueldeicaza reviewed Oct 20, 2014
View reviewed changes
mcs/class/System.IdentityModel/System.IdentityModel-net_4_5.csproj
</Target>
-->
<PropertyGroup>
<?xml version="1.0" encoding="utf-8"?>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Whitespace change, please resubmit once this is fixed, and squash the commits (so we do not end up nuking the file history with the two changes).

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Line endings corrected

@RyanMelenaNoesis
Additional JWT Security Token Support
950d9c8 
Fix line endings.  Add test for JavaScriptSerializer.Deserialize(string input, Type targetType).
migueldeicaza added a commit that referenced this pull request Oct 29, 2014
@migueldeicaza
Merge pull request #1337 from RyanMelenaNoesis/master
c265540 
Additional JWT Security Token Support
@migueldeicaza migueldeicaza merged commit c265540 into mono:master Oct 29, 2014
@akoeplinger
Copy link
Member

This breaks a bunch of tests:

MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.DeserializeDictionaryOfArrayList
MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.DeserializeObject
MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.InfinityAndNaN
MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.TestDeserialize
MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.TestDeserializeConverter1
MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.TestDeserializeNonGenericOverload
MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.TestDeserializeTypeResolver
MonoTests.System.Web.Script.Serialization.JavaScriptSerializerTest.TestSerialize1

@RyanMelenaNoesis
Copy link
Contributor Author

Unit Test issues corrected in PR #1380

picenka21 pushed a commit to picenka21/runtime that referenced this pull request Feb 18, 2022
@migueldeicaza
Merge pull request mono/mono#1337 from RyanMelenaNoesis/master
bbd2a12 
Additional JWT Security Token Support

Commit migrated from mono/mono@c265540
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@RyanMelenaNoesis @monoadmin @migueldeicaza @akoeplinger